Skip to content

devops: Add tests to release workflow before package creation#128

Merged
brandonh-msft merged 4 commits intomainfrom
copilot/fix-122
Aug 1, 2025
Merged

devops: Add tests to release workflow before package creation#128
brandonh-msft merged 4 commits intomainfrom
copilot/fix-122

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jul 31, 2025

This PR integrates automated testing into the release workflow to ensure code quality before creating and publishing NuGet packages.

Changes Made

Added a test step to the build-all-configs job in .github/workflows/release.yaml that runs between the build and pack operations:

- name: Test
  run: dotnet test --no-build --configuration ${{ matrix.configuration }} --framework net9.0

Impact

The release workflow now follows this sequence:

  1. Build - Compile code on all platforms (ubuntu, windows, macos) and configurations (Debug, Release)
  2. Test - Run all 247 tests to validate functionality ⬅️ NEW
  3. Pack - Create NuGet packages only if tests pass

This ensures that:

  • Only thoroughly tested code gets packaged and published to NuGet.org
  • Test failures will block the release process, preventing broken packages from being published
  • The existing dependency chain (build-package depends on build-all-configs) means failed tests will halt the entire release pipeline

The test command uses --no-build for efficiency since the code was just built, and --framework net9.0 to ensure compatibility with the CI environment.

Fixes #122.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: brandonh-msft <20270743+brandonh-msft@users.noreply.github.com>
Copilot AI changed the title [WIP] Run tests as part of release process Add tests to release workflow before package creation Jul 31, 2025
Copilot AI requested a review from brandonh-msft July 31, 2025 16:52
@brandonh-msft brandonh-msft marked this pull request as ready for review July 31, 2025 16:54
Run tests on all supported frameworks not just net9
@brandonh-msft brandonh-msft changed the title Add tests to release workflow before package creation devops: Add tests to release workflow before package creation Jul 31, 2025
@brandonh-msft brandonh-msft enabled auto-merge July 31, 2025 17:10
@brandonh-msft brandonh-msft removed their request for review July 31, 2025 17:10
@brandonh-msft brandonh-msft added this pull request to the merge queue Aug 1, 2025
Merged via the queue into main with commit ae1cfa8 Aug 1, 2025
4 checks passed
@brandonh-msft brandonh-msft deleted the copilot/fix-122 branch August 1, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run tests as part of release process

3 participants